feat(bun): upgrade to bun, reduce docker image size by 95%, upgrade docs & ci by waleedlatif1 · Pull Request #371 · simstudioai/sim

waleedlatif1 · 2025-05-18T07:45:06Z

migrate to bun
added envvars to drizzle
upgrade bun devcontainer feature to a valid one
added bun, docker not working
updated envvars, updated to bunder and esnext modules
fixed build, reinstated otel
feat: optimized multi-stage docker images
add coerce for boolean envvar
feat: add docker-compose configuration for local LLM services and remove legacy Dockerfile and entrypoint script
feat: add docker-compose files for local and production environments, and implement GitHub Actions for Docker image build and publish
refactor: remove unused generateStaticParams function from various API routes and maintain dynamic rendering
cleanup
upgraded bun
updated ci
fixed build

Description

Upgrade to bun, reduce docker image size by 95%, upgrade docs & ci

Type of change

New feature (non-breaking change which adds functionality)
Documentation update
Performance improvement
Code refactoring (no functional changes)

How Has This Been Tested?

Tested manually, ensured everything works.

Checklist:

My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have added tests that prove my fix is effective or that my feature works
All tests pass locally and in CI (npm test)
My changes generate no new warnings
Any dependent changes have been merged and published in downstream modules
I have updated version numbers as needed (if needed)
I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

My changes do not introduce any new security vulnerabilities
I have considered the security implications of my changes

Additional Information:

In a follow-up PR, we will implement the docker container using this new docker image and make the CLI to publish the new docker image.

…ove legacy Dockerfile and entrypoint script

… and implement GitHub Actions for Docker image build and publish

…I routes and maintain dynamic rendering

vercel · 2025-05-18T07:45:10Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs	🛑 Canceled (Inspect)			May 18, 2025 8:00am
sim	🛑 Canceled (Inspect)			May 18, 2025 8:00am

greptile-apps

PR Summary

This PR represents a comprehensive migration from Node.js/npm to Bun as the runtime environment and package manager, along with significant Docker configuration improvements and environment variable handling updates.

Migrated from npm/yarn to Bun, updating all package management commands, scripts, and configurations across the codebase
Introduced new multi-stage Docker builds with separate configurations for local development (docker-compose.local.yml) and production (docker-compose.prod.yml)
Added support for local LLM services through docker-compose.ollama.yml with both GPU and CPU configurations
Centralized environment variable handling through a new env.ts module using @t3-oss/env-nextjs for type-safe validation
Removed local storage mode functionality in favor of always using database synchronization, which could impact offline capabilities

_{137 file(s) reviewed, 47 comment(s)}
_{Edit PR Review Bot Settings | Greptile}

greptile-apps · 2025-05-18T07:45:46Z

+alias sim-start="cd /workspace && bun run dev"
+alias sim-migrate="cd /workspace/apps/sim && bunx drizzle-kit push"
+alias sim-generate="cd /workspace/apps/sim && bunx drizzle-kit generate"
+alias sim-rebuild="cd /workspace && bun run build && bun run start"


logic: sim-rebuild uses 'bun run start' but earlier aliases use 'bun run dev' - this inconsistency could cause confusion

greptile-apps · 2025-05-18T07:45:50Z

+RUN bun completions > /etc/bash_completion.d/bun
+
+# Set up shell environment
+RUN echo "export PATH=$PATH:/home/$USERNAME/.bun/bin" >> /etc/profile


logic: PATH modification doesn't include $USER_HOME variable, could break for different user configurations

Suggested change

RUN echo "export PATH=$PATH:/home/$USERNAME/.bun/bin" >> /etc/profile

RUN echo "export PATH=$PATH:$HOME/.bun/bin" >> /etc/profile

greptile-apps · 2025-05-18T07:45:50Z

+ARG USERNAME=bun
 ARG USER_UID=1000
 ARG USER_GID=$USER_UID


logic: User creation commands are missing - need to actually create the user before setting up sudo access

Suggested change

ARG USERNAME=bun

ARG USER_UID=1000

ARG USER_GID=$USER_UID

ARG USERNAME=bun

ARG USER_UID=1000

ARG USER_GID=$USER_UID

RUN groupadd --gid $USER_GID $USERNAME \

&& useradd --uid $USER_UID --gid $USER_GID -m $USERNAME

greptile-apps · 2025-05-18T07:46:12Z

+bun install || {
+  echo "⚠️ bun install had issues but continuing setup..."
 }


logic: Silently continuing after bun install failures could leave the environment in an inconsistent state. Consider failing fast here instead.

Suggested change

bun install || {

echo "⚠️ bun install had issues but continuing setup..."

}

bun install || {

echo "❌ bun install failed. Please check the errors above and try again."

exit 1

}

greptile-apps · 2025-05-18T07:46:13Z

+  for pkg in $(echo $NATIVE_DEPS | grep -oP '"[^"]*"' | tr -d '"' | grep -v "trustedDependencies"); do
+    echo "Checking compatibility for $pkg..."
+  done


logic: Native dependency check only logs but takes no action. Should verify/rebuild native dependencies for Bun compatibility.

greptile-apps · 2025-05-18T08:05:31Z

+COPY . .
+
+# Installing with full context to prevent missing dependencies error
+RUN bun install --omit dev --ignore-scripts


logic: redundant bun install - already copied node_modules from deps stage

greptile-apps · 2025-05-18T08:05:32Z

+
+# Required for standalone nextjs build
+WORKDIR /app/apps/sim
+RUN bun install sharp


style: sharp installation should be in deps stage to maintain proper layer caching

greptile-apps · 2025-05-18T08:05:34Z

+
+# Copy only package files needed for migrations
+COPY package.json bun.lock turbo.json ./
+COPY apps/sim/package.json ./apps/sim/db/


logic: Incorrect destination path for package.json - should be './apps/sim/' not './apps/sim/db/'

Suggested change

COPY apps/sim/package.json ./apps/sim/db/

COPY apps/sim/package.json ./apps/sim/

greptile-apps · 2025-05-18T08:05:35Z

+RUN bun install --omit dev --ignore-scripts && \
+    bun install --omit dev --ignore-scripts drizzle-kit drizzle-orm postgres next-runtime-env


style: Running bun install twice is inefficient - combine into single command with all packages

Suggested change

RUN bun install --omit dev --ignore-scripts && \

bun install --omit dev --ignore-scripts drizzle-kit drizzle-orm postgres next-runtime-env

RUN bun install --omit dev --ignore-scripts drizzle-kit drizzle-orm postgres next-runtime-env

greptile-apps · 2025-05-18T08:05:36Z

+COPY apps/sim/package.json ./apps/sim/package.json
+COPY apps/sim/lib/env.ts ./apps/sim/lib/env.ts
+
+WORKDIR /app/apps/sim


logic: No CMD or ENTRYPOINT specified for running migrations

waleedlatif1 and others added 14 commits May 17, 2025 17:11

migrate to bun

b4a2d85

added envvars to drizzle

e780fd8

upgrade bun devcontainer feature to a valid one

99e18b8

added bun, docker not working

7837c1e

updated envvars, updated to bunder and esnext modules

f74aadf

fixed build, reinstated otel

730bf83

feat: optimized multi-stage docker images

b1ccc86

add coerce for boolean envvar

df3e9d0

feat: add docker-compose configuration for local LLM services and rem…

db8e2f7

…ove legacy Dockerfile and entrypoint script

feat: add docker-compose files for local and production environments,…

c51b5e1

… and implement GitHub Actions for Docker image build and publish

refactor: remove unused generateStaticParams function from various AP…

f5bdd01

…I routes and maintain dynamic rendering

cleanup

d2ea965

upgraded bun

1ad8140

updated ci

37fbb12

waleedlatif1 changed the title ~~bunny~~ feat(bun): upgrade to bun, reduce docker image size by 95%, upgrade docs & ci May 18, 2025

waleedlatif1 force-pushed the bunny branch from f08cd42 to 24631b8 Compare May 18, 2025 07:50

vercel Bot deployed to Preview – docs May 18, 2025 07:51 View deployment

waleedlatif1 force-pushed the bunny branch 2 times, most recently from 013fcdf to 4994fa7 Compare May 18, 2025 07:53

fixed build

d9d82d8

waleedlatif1 force-pushed the bunny branch from 4994fa7 to d9d82d8 Compare May 18, 2025 07:53

vercel Bot deployed to Preview – docs May 18, 2025 07:56 View deployment

vercel Bot deployed to Preview – sim May 18, 2025 08:00 View deployment

waleedlatif1 merged commit 717e17d into main May 18, 2025

This was referenced May 18, 2025

feat: optimized multi-stage docker images (93% smaller) [WIP] #359

Closed

feat(bun): upgrade to bun, reduce docker image size by 95%, upgrade docs & ci #369

Closed

greptile-apps Bot reviewed May 18, 2025

View reviewed changes

waleedlatif1 deleted the bunny branch May 18, 2025 08:13

waleedlatif1 mentioned this pull request Jul 24, 2026

fix(deps): build isolated-vm on install via root trustedDependencies #5935

Merged

6 tasks

Navigation Menu

Uh oh!

feat(bun): upgrade to bun, reduce docker image size by 95%, upgrade docs & ci - #371

feat(bun): upgrade to bun, reduce docker image size by 95%, upgrade docs & ci#371
waleedlatif1 merged 15 commits into
mainfrom
bunny

waleedlatif1 commented May 18, 2025 •

edited

Loading

Uh oh!

vercel Bot commented May 18, 2025 •

edited

Loading

Uh oh!

greptile-apps Bot left a comment

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

greptile-apps Bot May 18, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	RUN echo "export PATH=$PATH:/home/$USERNAME/.bun/bin" >> /etc/profile
	RUN echo "export PATH=$PATH:$HOME/.bun/bin" >> /etc/profile

	COPY apps/sim/package.json ./apps/sim/db/
	COPY apps/sim/package.json ./apps/sim/

		RUN bun install --omit dev --ignore-scripts && \
		bun install --omit dev --ignore-scripts drizzle-kit drizzle-orm postgres next-runtime-env

Uh oh!

Conversation

waleedlatif1 commented May 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of change

How Has This Been Tested?

Checklist:

Security Considerations:

Additional Information:

Uh oh!

vercel Bot commented May 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

greptile-apps Bot left a comment

Choose a reason for hiding this comment

PR Summary

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps Bot May 18, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

waleedlatif1 commented May 18, 2025 •

edited

Loading

vercel Bot commented May 18, 2025 •

edited

Loading